-
Notifications
You must be signed in to change notification settings - Fork 105
[WIP] Online Batch Support #490
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Shimi Bandiel <[email protected]>
ahg-g
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @shimib , can we add a user guide to clearly show how everything fits together?
test/integration/redisimpl_test.go
Outdated
| ) | ||
|
|
||
| const ( | ||
| redisURL = "localhost:6379" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, will do
| { | ||
| "id" : "19933123533434", | ||
| "deadline" : "1764045130", | ||
| "payload": {"model":"food-review","prompt":"hi", "max_tokens":10,"temperature":0} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shimib This only supports /v1/completions. Is that on purpose?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It shouldn't, endpoint will be configurable. Will be in the user guide
upload image Signed-off-by: Shimi Bandiel <[email protected]>
|
🚨 Unsigned commits detected! Please sign your commits. For instructions on how to set up GPG/SSH signing and verify your commits, |
…d Makefile support for batch exec. Signed-off-by: Shimi Bandiel <[email protected]>
API and initial implementation for online batch processing (asynch processing).